Skip to content

add haven storage window object and extension detection#6

Open
thisisommore wants to merge 50 commits intoxxfoundation:mainfrom
thisisommore:5-add-window-object-and-extension-detection
Open

add haven storage window object and extension detection#6
thisisommore wants to merge 50 commits intoxxfoundation:mainfrom
thisisommore:5-add-window-object-and-extension-detection

Conversation

@thisisommore
Copy link

@thisisommore thisisommore commented Aug 4, 2025

Changes

1. Use extension storage

  • Added use of extension storage
  • Using port from chrome runtime to communicate with extension

localStorage is still used

  • If extension is not available
  • If user have there keys in localStorage

2. Use promise for some functions

  • Storage operation with extension is asynchronous
  • Some functions using this storage operation where causing blocking issue
  • Without promise the go thread blocks main thread and therefore the response from extension stays in event loop
  • With promise the go thread doesn't block main thread and allows response from extension to be processed

Extension

Note

Window object is created on haven side, not on extension side.
This allows us to keep permission for extension limited to storage only.

… new repository for speakeasy-web"

This reverts commit 36bff9c.
…Storage and remove unnecessary console logs in extSCallback.
…r wasmbrowsertest and cleanenv, and set environment variables for Go WebAssembly build.
…est and cleanenv, and rename wasmbrowsertest binary for Go WebAssembly compatibility.
…env from .gitpod.yml, and rename wasmbrowsertest binary for improved Go WebAssembly compatibility.
…and improve configuration management. Removed unnecessary xxdk-wasm checkout location and added global git configuration handling in Dockerfile.
@thisisommore thisisommore marked this pull request as ready for review August 5, 2025 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant